Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused imports in Java and xtend classes #201

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

HannesWell
Copy link
Contributor

Run the 'Organize imports' action on each class that has unused imports.

On some classes the re-order also existing and used imports. Do you have specific settings for the order of import statements (e.g. in Eclipse preferences: Java -> Code Style -> Organize Imports)? I just used the default settings for this. Please let me know if I should change them.

Run the 'Organize imports' action on each class that has unused imports.
@HannesWell HannesWell force-pushed the remove-unused-imports branch from a68cc2e to 02c6a83 Compare August 13, 2024 11:30
Copy link
Member

@NiklasRentzCAU NiklasRentzCAU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Our ordering is as suggested by you, just by applying the auto-organize from Eclipse. Some of these files however were auto-generated by Xtext (version 2.10 according to the sources) and therefore might change back when re-generating. But as we have not re-generated these files for a long time and do not plan to, and because Xtext had a lot of updates since then, I will just approve these changes as well for reducing the amount of warnings. Thank you!

@NiklasRentzCAU NiklasRentzCAU added the cleanup This issue or PR has no own functionality, but cleans up the code base. label Aug 14, 2024
@NiklasRentzCAU NiklasRentzCAU merged commit 85cedde into kieler:master Aug 14, 2024
2 checks passed
@HannesWell HannesWell deleted the remove-unused-imports branch August 14, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup This issue or PR has no own functionality, but cleans up the code base.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants